This feels like a reasonable approach. I'm not convinced on the name, perhaps something like disabledInstrumentations or suppressedInstrumentation is more descriptive?

I didn't include the word 'instrumentation' in the name because it's already scoped to the 'instrumentations' block....which would make it redundant.

instrumentations {
  suppressing (
    "unwanted.instrumentation.name",
    "something.unwanted"
  )
  ...
}

I think I see where you're coming from, though. Nothing else (yet?) in the DSL uses the infinitive verb form (tho it's not super uncommon in other DSLs I think). Maybe you would like suppress or suppressed better? I'm open to changing it.

While reviewing I also noticed that OtelRumConfig has shouldDiscoverInstrumentations(). I'd be inclined to keep the scope of this PR tight, but whether we need something similar should be part of related discussion IMO.

Oh that definitely occurred to me as well, and we should add that....agree not in this PR.

Created #1716 to follow up on this idea.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.10%. Comparing base (20e3cc1) to head (286904f).
⚠️ Report is 18 commits behind head on main.

@@            Coverage Diff             @@
##             main    #1714      +/-   ##
==========================================
- Coverage   62.41%   62.10%   -0.32%     
==========================================
  Files         160      159       -1     
  Lines        3480     3446      -34     
  Branches      353      347       -6     
==========================================
- Hits         2172     2140      -32     
+ Misses       1209     1206       -3     
- Partials       99      100       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.